-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix buildkite run so that unit tests run on GPU #739
base: main
Are you sure you want to change the base?
Conversation
bf8038c
to
309d6f3
Compare
309d6f3
to
74e15a2
Compare
@charleskawczynski do you have any ideas about the bug up above? I can reproduce the error, and I tried added more parametric types to the
Im trying to use @code_warntype in the REPL but am not clearly seeing any "Any" pop up:
|
I pushed a commit! I think the issue was that we were reaching into global scope for the default argument |
@charleskawczynski it looks like this is still failing - do you have any other ideas? |
I am wondering if this will work after Teja's PR refactoring the rooting distribution? |
I'm not sure if this is the problem, but |
Also, for consistency, I suggest changing |
Purpose
Some of our buildkite runs were not running on GPU. Update buildkite so that they are.
To-do
experiments/integrated/performance/profile_allocations.jl
. Note that the same code runs on the Clima cluster on GPU. This is the error message:/central/home/kdeck/ClimaLand.jl/src/standalone/Soil/soil_heat_parameterizations.jl:65:
Called by:
Note that
typeof(hydrology_cm) <: AbstractSoilHydrologyClosure
and we have setBase.broadcastable(x::AbstractSoilHydrologyClosure) = tuple(x)
Content